Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Dependencies for Enhanced Compatibility with Newer Python Versions #5

Merged
merged 61 commits into from
Apr 21, 2024

Conversation

vincentadam87
Copy link
Collaborator

@vincentadam87 vincentadam87 commented Apr 10, 2024

Configuration changes in pyproject.toml

  1. cmake: ">=3.18.0,<3.19.0" from "~3.18.0"
  2. importlib-metadata: ">=4.4,<5.0" from "importlib_metadata", "^1.6"
  3. numpy: ">=1.18.0,<2.0.0" from "^1.18.0"
  4. python: ">=3.7,<3.12" from "~3.7"
  5. tensorflow:
    • Python 3.7 to 3.8: ">=2.4.0,<2.7.0"
    • Python 3.9: ">=2.5.0,<2.7.0"
    • Python 3.10: ">=2.7.0,<2.9.0"
    • Python 3.11: ">=2.8.0,<2.16.0"
  6. mypy:
    • Python 3.7 to 3.8: ">=1.1.0,<1.3.0"
    • Python 3.9: ">=1.1.0,<1.3.0"
    • Python 3.10: ">=1.1.0,<1.3.0"
    • Python 3.11: "1.8.0"
  7. pytest: Updated to "6.2.5" from "^5.3.5"
  8. pytest-black: Updated to ">=0.3.8" from "^0.3.8"

Gihub workflows changes

  1. Expanded Python Versions: Added Python 3.8.12, 3.9.12, and 3.10.4 to the matrix.
  2. Updated Poetry Version: Upgraded from 1.1.6 to 1.1.12.
  3. Updated Build Tools: Installed latest build tools including g++-14.
  4. New C++ Build Steps: Added steps for configuring and building a C++ library with CMake.

Copy link

@johnamcleod johnamcleod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment on lines +61 to +63
mkdir -p banded_matrices/build
cd banded_matrices/build
cmake .. -DCMAKE_BUILD_TYPE=Release

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.github/workflows/quality-check.yaml Outdated Show resolved Hide resolved
Comment on lines +46 to +48
- name: Configure environment variables for CMake
run: |
echo "PYTHON_BIN=$(which python)" >> $GITHUB_ENV

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be better to define the environment variable in a build step specifically: https://docs.github.com/en/actions/learn-github-actions/variables#defining-environment-variables-for-a-single-workflow

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure what you mean here!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than having a separate step to define a new environment variable and piping it into the global environment, the link describes how to add an env statement to the step in the github action definition for defining contextual environment variables.

pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
@vincentadam87
Copy link
Collaborator Author

Regarding the workflow, I have a question
A lot of the novelty in this PR is about doing some mkdir and cd at the right place so that the compiled files may be found.
This complexifies the workflows.
Is there a way to edit the build.py so that this is done there?

@vincentadam87 vincentadam87 changed the title Va/update bm Update Dependencies for Enhanced Compatibility with Newer Python Versions Apr 21, 2024
@vincentadam87 vincentadam87 merged commit db0ef35 into master Apr 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants